Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[Feature/podaac-6307] Adjust authentication to use earthaccess instead of calling urs.earthdata api directly #182

Closed
wants to merge 11 commits into from
Closed

[Feature/podaac-6307] Adjust authentication to use earthaccess instead of calling urs.earthdata api directly #182

wants to merge 11 commits into from

Conversation

hkryeung
Copy link

Removed token generation in favor to using the earthaccess library: https://github.com/nsidc/earthaccess
This update forces python to be 3.10+ so poetry.lock is updated accordingly

For tests, most of it is not needed since access is managed by earthaccess (instead of the cookie section setup_earthdata_login_auth which needs to be kept)

@hkryeung
Copy link
Author

hkryeung commented Mar 13, 2025 •
edited
Loading

All the failed builds are sourced from .netrc file (specifically the windows _netrc variant) either not being generated or earthaccess isn't working as expected; and sadly I don't have a Windows Machine to debug

One way to bypass the _netrc file generation would to be add in environment variables EARTHDATA_USERNAME and EARTHDATA_PASSWORD, to allow earthaccess to gain access, but it might be a security issue

@mike-gangl any suggestions on how to debug without a windows machine?

@hkryeung hkryeung closed this Mar 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@hkryeung